-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(Ready for review and merge) Add Chat Rewind, Duplicate, and Fork Features #305
(Ready for review and merge) Add Chat Rewind, Duplicate, and Fork Features #305
Conversation
There was a discussion about a feature like this earlier: #114 the proposal was to use isomorphic-git. But that requires a browserFS. This approch doesn't add to much overhead and makes it easy to move back in time. Although what happens when you make changes, the downstream chat messages are not aware of that, Or am |
Currently downstream chats will be overwritten after you make a new chat message call. As for referenced PR. It does way more and differently, and I am not sure AI is aware of those manipulations, I don't see it changing messages. In that sense proposal here seems to be more in line with how Bolt.New works, is smaller and more elegant in that way. But does not yet do anything to import/export to git. |
I was also looking at this one this morning I see them both as powerful features that will likely get merged in pretty quick. |
@Digitl-Alchemyst thanks for feedback, I will take a look at better icon and hint on hover. I will be adding few more things after discussion in community. |
This is really clean and good. Just one point to note, there's an existing issue I'm not sure we've captured where a revisited chat doesn't behave like the first run: For React at least, first run generates package.json and will not run the npm install until it's complete. On revisit, it doesn't care and runs both in tandem; in that case the install step always fails. More making a note for myself, that isn't related to this change so I think this is worth bringing in for now. I'll look for/create an issue related to that behavior shortly. 👍 from me to merge |
Ok, merging then |
Hey @wonderwhy-er |
You did not describe the issue, chat history is gone when you restart server? |
No it is running without any restart. |
Weird, works fine for me in hosted variant Are you using cancary or may be incognito? |
No I am deploying with docker on coolify. |
would you be willing to rebase these changes in the (almost original) bolt - https://github.com/Bolt-CE/bolt |
Video
2024-11-18.21-25-34.mp4
Summary
This PR introduces several new features:
Dependency Updates
@radix-ui/react-tooltip
for improved tooltip functionality.pnpm-lock.yaml
to ensure compatibility and resolve dependency conflicts.